-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate from GMaven to GMavenPlus #1745
Migrate from GMaven to GMavenPlus #1745
Conversation
Works perfectly on JDK11! |
Ah, interesting. I was getting some weird Mockito errors I didn't understand on Java 8. |
No I'm sorry, It doesn't work like a charm. Mvn clean package does work in fact. But mvn clean install throws the same errors on the integration tests as I've written in my pull request. IMO some groovy exception throwing is different once switching to gmavenplus....? |
Checkout master, build with JDK 8, works Any guidance? |
I don't understand yet why changing plugins would cause a change in behavior. The only thing I can guess is that there's different versions of Groovy dependencies on the classpath during compilation, but I haven't found them yet. |
Hi @keeganwitt did you figure anything out or have any ideas on how to figure it out? If not I'm kinda stuck |
Not yet. I've reached out to the other Groovy devs to see if they have any ideas. This seems very strange. |
91e8708
to
a31ba29
Compare
I can't even build the main branch on Windows, even when using Docker like docker run --rm -v "${HOME}/.m2:/root/.m2" -v "${pwd}:/project" -w /project maven:3.8-eclipse-temurin-8 mvn clean test I get multiple package does not exist and "cannot find symbol" compilation errors. |
@keeganwitt Im no windhoos user so I can't guide you 🥺 |
What's strange is I'm getting lots of errors like
And neither JsonSchemaFactory or JsonSchemaValidator are Groovy classes. |
5bcb4d6
to
023c9a5
Compare
Integration tests are failing, but compilation is succeeding. |
What's the status here? Can we get the tests to work? I would love to make this switch. |
I'm having trouble understanding why it is the tests are failing. I need to understand the tests better I guess. |
@johanhaleby can you take a look and explain the tests that are failing. Or determine whether they fail as expected or because of the gmaven upgrade and wether we should change the tests accordingly. |
I'm not sure why. But the tests on this branch pass when I run them in an Ubuntu VM. It appears the tests require they be run on Linux. |
I created #1757 to ensure the changes run successfully before merging. |
Unfortunately I have to revert his because lots of tests are failing |
This reverts commit b653f7a.
Splitting the migration to GMavenPlus in #1742 into a separate PR.